Skip to content

fix: import skills to .copilot/skills/ instead of .ai-team/skills/ - #849

Merged
bradygaster merged 2 commits into
devfrom
squad/fix-skills-import-path
Apr 11, 2026
Merged

bradygaster merged 2 commits into
devfrom
squad/fix-skills-import-path

Conversation

@bradygaster

Copy link
Copy Markdown
Collaborator

Skills import was writing to .ai-team/skills/ but the canonical location is .copilot/skills/. This fixes the last 6 test failures blocking the insider release.

Skills canonical location is .copilot/skills/, not .ai-team/skills/.
Import was writing to the wrong path.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 5, 2026 09:12
@github-actions

github-actions Bot commented Apr 5, 2026

Copy link
Copy Markdown
Contributor

🟢 Impact Analysis — PR #849

Risk tier: 🟢 LOW

📊 Summary

Metric Count
Files changed 1
Files added 0
Files modified 1
Files deleted 0
Modules touched 1

🎯 Risk Factors

  • 1 files changed (≤5 → LOW)
  • 1 module(s) touched (≤1 → LOW)

📦 Modules Affected

root (1 file)
  • index.cjs

This report is generated automatically for every PR. See #733 for details.

@github-actions

github-actions Bot commented Apr 5, 2026

Copy link
Copy Markdown
Contributor

🛫 PR Readiness Check

ℹ️ This comment updates on each push. Last checked: commit 878277e

⚠️ 2 item(s) to address before review

Status Check Details
✅ Single commit 1 commit — clean history
✅ Not in draft Ready for review
✅ Branch up to date Up to date with dev
❌ Copilot review No Copilot review yet — it may still be processing
✅ Changeset present No source files changed — changeset not required
✅ Scope clean No .squad/ or docs/proposals/ files
✅ No merge conflicts No merge conflicts
✅ Copilot threads resolved No Copilot review threads
❌ CI passing 12 check(s) still running

This check runs automatically on every push. Fix any ❌ items and push again.
See CONTRIBUTING.md and PR Requirements for details.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes the squad import command to write imported skills into the canonical Copilot skills location (.copilot/skills/) rather than the legacy .ai-team/skills/, aligning import behavior with existing export behavior and unblocking failing tests.

Changes:

  • Update import-time skills destination to .copilot/skills/<skill>/SKILL.md.
  • Add a clarifying comment indicating .copilot/skills/ is the canonical location.

Comment thread index.cjs Resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@bradygaster
bradygaster merged commit cf8ffdc into dev Apr 11, 2026
tamirdresher pushed a commit that referenced this pull request Apr 21, 2026
)

* fix: import skills to .copilot/skills/ instead of .ai-team/skills/

Skills canonical location is .copilot/skills/, not .ai-team/skills/.
Import was writing to the wrong path.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Apply suggestion from @Copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@bradygaster
bradygaster deleted the squad/fix-skills-import-path branch August 8, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants